Skip to content

Reader: Disable recommended tags in Discover #25018

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Nov 26, 2025
Merged

Reader: Disable recommended tags in Discover #25018

merged 2 commits into from
Nov 26, 2025
+4 −6

Conversation

@kean
Copy link
Contributor

@kean kean commented Nov 25, 2025 •
edited
Loading

Fixes CMM-976: Be more selective about "You might like" topics (see ticket for rationale)

I kept the presentation code because there are still cards on disk. The remaining files can be removed later.

@kean kean added this to the 26.6 milestone Nov 25, 2025
@kean kean added the Reader label Nov 25, 2025
@kean kean requested a review from jkmassel November 25, 2025 12:38
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Nov 25, 2025 •
edited
Loading

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number29934
VersionPR #25018
Bundle IDcom.jetpack.alpha
Commitac1bcea
Installation URL358ps4p0bb6l0
 Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Nov 25, 2025 •
edited
Loading

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number29934
VersionPR #25018
Bundle IDorg.wordpress.alpha
Commitac1bcea
Installation URL6kkogo5v27htg
 Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@kean kean force-pushed the fix/cmm-976-recommended-tags-remove branch from fc1d9c4 to 696085c Compare November 26, 2025 00:22
@kean kean changed the base branch from trunk to release_notes/26.5 November 26, 2025 00:22
@kean kean requested a review from crazytonyli November 26, 2025 00:23
@kean
Copy link
Contributor Author

kean commented Nov 26, 2025

I got a seal of approval to remove it.

@dangermattic
Copy link
Collaborator

dangermattic commented Nov 26, 2025 •
edited
Loading

1 Warning
⚠️ This PR is assigned to the milestone 26.5 ❄️. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

@kean kean changed the base branch from release_notes/26.5 to release/26.5 November 26, 2025 00:23
@kean kean modified the milestones: 26.6, 26.5 ❄️ Nov 26, 2025
@kean kean enabled auto-merge (squash) November 26, 2025 12:39
@kean kean force-pushed the fix/cmm-976-recommended-tags-remove branch from 0759c02 to ac1bcea Compare November 26, 2025 13:14
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 26, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@kean kean merged commit 3319f60 into release/26.5 Nov 26, 2025
26 of 32 checks passed
@kean kean deleted the fix/cmm-976-recommended-tags-remove branch November 26, 2025 13:30
jkmassel pushed a commit that referenced this pull request Nov 27, 2025
@kean @jkmassel
Reader: Disable recommended tags in Discover (#25018)
432c676 
* Disable recommended tags in Discover

* Update unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@crazytonyli crazytonyli crazytonyli approved these changes

@jkmassel jkmassel Awaiting requested review from jkmassel

Assignees

No one assigned

Labels

Reader

Projects

None yet

Milestone

26.5 ❄️

Development

Successfully merging this pull request may close these issues.

5 participants

@kean @wpmobilebot @dangermattic @crazytonyli